Skip to content

RUN-4569 Fix git-plugin POM developers for Maven Central#50

Merged
fdevans merged 1 commit into
mainfrom
RUN-4569-git-plugin-pom-developers
Jul 10, 2026
Merged

RUN-4569 Fix git-plugin POM developers for Maven Central#50
fdevans merged 1 commit into
mainfrom
RUN-4569-git-plugin-pom-developers

Conversation

@fdevans

@fdevans fdevans commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix the <developers> section being omitted from the published POM, which caused closeSonatypeStagingRepository to fail Central validation (400 - Deployment reached an unexpected status: Failed).
  • Root cause: gradle/publishing.gradle used project.ext.hasProperty('developers'), which checks the ExtraPropertiesExtension object for a JavaBean property named developers (always false) instead of the dynamic ext property. Changed to if (project.ext.developers), matching the shared/salt-step publishing script.

Verification

  • generatePomFileForGit-pluginPublication now emits <developers> with ltamaster (Luis Toledo).

Follow-up

  • After merge, re-tag 2.0.1 on the fixed commit to re-run the release + Central publish.

publishing.gradle used project.ext.hasProperty('developers'), which checks the ExtraPropertiesExtension for a JavaBean property (always false) rather than the dynamic ext property, so the <developers> block was omitted. Maven Central requires developers, so closeSonatypeStagingRepository failed validation. Match the shared/salt-step form: if (project.ext.developers).
@fdevans fdevans requested review from a team and Copilot July 10, 2026 18:55
@fdevans fdevans merged commit 943e6b2 into main Jul 10, 2026
3 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes Maven POM metadata publication for the git-plugin by ensuring the <developers> section is emitted, addressing Maven Central validation failures during closeSonatypeStagingRepository.

Changes:

  • Update the conditional check in the publishing POM configuration to correctly detect and include ext.developers.
  • Ensure generatePomFileForGit-pluginPublication includes the expected <developers> entries when ext.developers is configured.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants